Skip to content

Clean up IBKR fractional sizing support - #49

Merged
Pigbibi merged 3 commits into
mainfrom
codex/review-fractional-cleanups
May 6, 2026
Merged

Clean up IBKR fractional sizing support#49
Pigbibi merged 3 commits into
mainfrom
codex/review-fractional-cleanups

Conversation

@Pigbibi

@Pigbibi Pigbibi commented May 6, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • share quantity-step env parsing through QuantPlatformKit and update the pin
  • use position quantity for zero-target sells to avoid residual shares from quote drift
  • document IBKR fractional-share runtime env vars and precedence

Tests

  • ./.venv/bin/python -m pytest tests/test_runtime_config_support.py tests/test_execution_service.py::test_execute_rebalance_can_submit_fractional_buy_when_quantity_step_allows tests/test_execution_service.py::test_execute_rebalance_zero_target_sell_uses_position_quantity

Holding merge until CI and Codex review comments are addressed.

@Pigbibi
Pigbibi merged commit 05103e0 into main May 6, 2026
1 check passed
@Pigbibi
Pigbibi deleted the codex/review-fractional-cleanups branch May 6, 2026 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant